Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

src: Fix "no-unnecessary-condition" issues (HMS-5355) #2760

Merged

Conversation

regexowl
Copy link
Collaborator

@regexowl regexowl commented Jan 17, 2025

Rebased on #2755

This is a first batch of "no-unnecessary-condition" issue fixes.

JIRA: HMS-5355

@regexowl
Copy link
Collaborator Author

Can squash the commits later, needed to keep them separate to not get lost in the changes.

Copy link

codecov bot commented Jan 17, 2025

Codecov Report

Attention: Patch coverage is 77.77778% with 4 lines in your changes missing coverage. Please review.

Project coverage is 84.76%. Comparing base (1c50407) to head (65984a8).
Report is 5 commits behind head on main.

Files with missing lines Patch % Lines
src/Components/ImagesTable/Instance.tsx 33.33% 2 Missing ⚠️
src/Components/Blueprints/BuildImagesButton.tsx 75.00% 1 Missing ⚠️
src/Components/LandingPage/LandingPage.tsx 0.00% 1 Missing ⚠️

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2760      +/-   ##
==========================================
- Coverage   84.77%   84.76%   -0.01%     
==========================================
  Files         187      187              
  Lines       21298    21295       -3     
  Branches     2099     2099              
==========================================
- Hits        18055    18051       -4     
- Misses       3221     3222       +1     
  Partials       22       22              
Files with missing lines Coverage Δ
src/Components/Blueprints/BlueprintActionsMenu.tsx 62.50% <ø> (-0.58%) ⬇️
src/Components/Blueprints/BlueprintsPagination.tsx 79.06% <100.00%> (ø)
src/Components/Blueprints/BlueprintsSideBar.tsx 86.70% <100.00%> (ø)
src/Components/Blueprints/DeleteBlueprintModal.tsx 79.24% <100.00%> (ø)
...ueprints/helpers/onPremToHostedBlueprintMapper.tsx 88.15% <100.00%> (ø)
src/Components/ImagesTable/ImagesTable.tsx 92.19% <100.00%> (ø)
src/Components/ImagesTable/ImagesTableToolbar.tsx 100.00% <100.00%> (ø)
src/Components/ImagesTable/Status.tsx 87.81% <100.00%> (ø)
src/Components/ImagesTable/Target.tsx 100.00% <100.00%> (ø)
src/Utilities/useDebounce.tsx 100.00% <100.00%> (ø)
... and 4 more

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1c50407...65984a8. Read the comment docs.

@regexowl regexowl force-pushed the fix-no-unnecessary-condition-issues branch from 4b3239b to f679fab Compare January 20, 2025 07:50
@regexowl regexowl marked this pull request as ready for review January 20, 2025 07:50
@regexowl
Copy link
Collaborator Author

/jira-epic HMS-5173

@schutzbot schutzbot changed the title src: Fix "no-unnecessary-condition" issues src: Fix "no-unnecessary-condition" issues (HMS-5355) Jan 20, 2025
@regexowl regexowl force-pushed the fix-no-unnecessary-condition-issues branch 2 times, most recently from 340f8b9 to 864ad43 Compare January 23, 2025 08:11
@croissanne croissanne force-pushed the fix-no-unnecessary-condition-issues branch from 864ad43 to 2dc7918 Compare January 23, 2025 12:31
@regexowl regexowl force-pushed the fix-no-unnecessary-condition-issues branch from 2dc7918 to fc85d85 Compare January 23, 2025 13:39
@regexowl
Copy link
Collaborator Author

/retest

Copy link
Collaborator

@lucasgarfield lucasgarfield left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice!

@lucasgarfield lucasgarfield force-pushed the fix-no-unnecessary-condition-issues branch from fc85d85 to 65984a8 Compare January 25, 2025 23:33
@lucasgarfield lucasgarfield merged commit 199dd3d into osbuild:main Jan 26, 2025
16 of 18 checks passed
@regexowl regexowl deleted the fix-no-unnecessary-condition-issues branch January 27, 2025 07:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants